Skip to content

fix(onboarding): gate dashboard escape hatch by auth state#3045

Merged
Drixled merged 9 commits intomainfrom
fix/get-started-dashboard-link
May 7, 2026
Merged

fix(onboarding): gate dashboard escape hatch by auth state#3045
Drixled merged 9 commits intomainfrom
fix/get-started-dashboard-link

Conversation

@Drixled
Copy link
Copy Markdown
Contributor

@Drixled Drixled commented May 5, 2026

Summary

  • Adds a /get-started dashboard escape hatch for signed-in users so they can skip the starting-point chooser and land on /profile.
  • Hides that escape hatch for signed-out visitors, leaving only the existing Already have an account? Sign in path so the footer does not present two competing mini-actions.
  • Adds focused end-to-end coverage for both auth states on /get-started. Architectural changes: none; this stays within the existing onboarding page component and its Playwright spec.

Verification

  • Not manually verified in a browser during this session; this was a small auth-state footer change and I relied on focused automated coverage while working.
  • Add any additional manual verification details before merge.

Visual Changes

Before After
Signed-out /get-started footer showing both Skip to dashboard and Sign in. Signed-out /get-started footer showing only Sign in.
Signed-in /get-started footer without a clear dashboard escape hatch. Signed-in /get-started footer showing Skip to dashboard.

Reviewer Notes

  • Review both /get-started auth states: signed-out users should only see the sign-in prompt, while signed-in users should only see the dashboard escape hatch.
  • /profile remains the escape hatch target to match the existing dashboard/profile landing route.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 5, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/app/get-started/personal/_components/ProductOptionsContent.tsx
  • apps/web/tests/e2e/get-started-dashboard-link.spec.ts

Reviewed by gpt-5.5-2026-04-23 · 681,871 tokens

@Drixled Drixled requested a review from lambertjosh May 5, 2026 16:07
@lambertjosh
Copy link
Copy Markdown
Contributor

Hrm - I'm not seeing it when auth'd?

image

@Drixled Drixled changed the title fix(onboarding): add dashboard escape hatch fix(onboarding): gate dashboard escape hatch by auth state May 7, 2026
@Drixled Drixled merged commit 0c26b86 into main May 7, 2026
13 checks passed
@Drixled Drixled deleted the fix/get-started-dashboard-link branch May 7, 2026 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants